Skip to content

Add unit & end-to-end tests for instancedMesh2 features#153

Open
Glavin001 wants to merge 6 commits into
agargaro:masterfrom
Glavin001:feat/tests
Open

Add unit & end-to-end tests for instancedMesh2 features#153
Glavin001 wants to merge 6 commits into
agargaro:masterfrom
Glavin001:feat/tests

Conversation

@Glavin001
Copy link
Copy Markdown

No description provided.

…E tests for frustum culling, LOD switching, raycasting, and rendering.

Update package dependencies and configurations for testing. Add test setup utilities and initial test cases for dynamic capacity and visibility features.
…to improve clarity and performance. Remove unnecessary texture initialization calls and enhance assertions for texture existence and data validation.
@Glavin001 Glavin001 changed the title Add GitHub Actions workflow for automated testing and update .gitignore to include test artifacts Add unit & end-to-end tests for instancedMesh2 features Dec 4, 2025
…ing, LOD switching, and rendering. Enhance test clarity by reducing code duplication and improving assertions. Introduce test helpers for scene setup and instance management.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 4, 2025

Quality Gate Passed Quality Gate passed

Issues
249 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.6% Duplication on New Code

See analysis details on SonarQube Cloud

@Glavin001 Glavin001 mentioned this pull request Dec 4, 2025
9 tasks
@Glavin001
Copy link
Copy Markdown
Author

I don't know how to tell SonarQube to ignore this these. Makes no sense to add more complexity to the test files for this Math.random() lint complaint.

@agargaro
Copy link
Copy Markdown
Owner

agargaro commented Dec 4, 2025

I should disable that rule, it's pretty useless on this repository. Don't worry and thank you!

Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants